-
Notifications
You must be signed in to change notification settings - Fork 99
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add set_wlan_conf #45
base: master
Are you sure you want to change the base?
Conversation
Please include README documentation. Thanks! |
Done. |
This seems like a very specific, personal use case; to update the password and SSID. This method should be fully fleshed out IMO to be able to set all wlan configurations (ie. Or an alternative, create specific methods such as Also, partially duplicate of #34 |
@maxirus I was modeling this after the basic settings on the sloofmaster PHP API client. I.e. basic WLAN settings. And name and password definitely fit into the "basic settings" nomenclature. @finish06 I'm happy to, if you want me to incorporate everything into one function that may be too much and we may want more specialized functions for certain things. An example would be disable, delete, mac_filter etc. Let me know the requirements and I'll update this one to add more. Alternatively, let me know if you just want me to go through all the settings a propose what should be in a main function and what is probably better broken out. I'm committed to this as I'm running a product in production that relies on this library. |
@tertius I am comfortable with either approach, i.e. a function to rule all WLAN configuration or functions calls that specify what is being accomplished. Depending on your Python exposure, I think the latter would be easier without resulting in any regressions or issues, etc. |
Is there any progress on this one? Or do we need another one to pick up on it, and finish it? If needed, I would be happy to join in on the development.. |
@finish06 Any news on this (rather old) PR? Should I do a refactoring of it, so it can be merged? (since there are conflicts) |
Feel free to take over. |
Adds the ability to change/set the WLAN config.